-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat : 대학/학과 공지 뷰 api 연결 및 무한 스크롤 적용 #167
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
관련 지식이 모자라서 적극적인 리뷰가 어려운 점 양해부탁드립니다ㅠㅠ
전체적인 코드 정렬과 id 컨벤션 수정 정도면 될거 같아용
코멘트 확인 부탁드립니다!
app/src/main/java/com/dongyang/android/youdongknowme/data/repository/NoticeRepository.kt
Show resolved
Hide resolved
app/src/main/java/com/dongyang/android/youdongknowme/standard/network/RetrofitObject.kt
Show resolved
Hide resolved
app/src/main/java/com/dongyang/android/youdongknowme/ui/view/notice/NoticeFragment.kt
Show resolved
Hide resolved
override fun onResume() { | ||
super.onResume() | ||
val intentFilter = IntentFilter(ACTION.FCM_ACTION_NAME) | ||
LocalBroadcastManager.getInstance(requireContext()) | ||
.registerReceiver(localBroadCast, intentFilter) | ||
} | ||
|
||
override fun onStop() { | ||
super.onStop() | ||
LocalBroadcastManager.getInstance(requireContext()).unregisterReceiver(localBroadCast) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
단순 궁금증 : 이 파트가 왜 지워진건지 궁금합니당
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fcm 관련 코드인 것 같아서 지웠는데 fcm 제가 구현해보고 싶어서요 ㅎㅅㅎ
app/src/main/java/com/dongyang/android/youdongknowme/ui/view/notice/NoticeFragment.kt
Show resolved
Hide resolved
app/src/main/java/com/dongyang/android/youdongknowme/ui/view/notice/NoticeViewModel.kt
Outdated
Show resolved
Hide resolved
06f0379
to
52dec17
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨어요~!👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
중요한 부분이라 늦었지만 리뷰 남겨봅니다.
- 다크모드 혹은 가로모드 변경으로 구성 변경을 발생시킨 후 페이징이 정상적으로 작동하는지 테스트 해보아요 :)
- 일반적으로 페이징의 경우 전체 로딩을 띄우기 보단 리스트 마지막에 로딩을 띄우는 경우가 많습니다! 이 부분 도전해보시면 어떨까요?
|
||
NoticeTabType.FACULTY -> viewModelScope.launch { | ||
when (val result = noticeRepository.fetchDepartmentNotices( | ||
"컴퓨터소프트웨어공학과", DEFAULT_REFRESH_PAGE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
임시 코드일까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네 아직 학과 설정 부분 연결이 되지 않아서 추후에 수정할 예정입니다
📮 관련 이슈
✍️ 구현 내용
데이터 바인딩
을 제거하고뷰바인딩
적용했습니다.📷 구현 영상
Screen_recording_20240223_220356.webm
✔️ 확인 사항